From: Isaku Yamahata <yamahata@valinux.co.jp>
Date: Fri, 3 Oct 2008 03:24:49 +0000 (+0900)
Subject: [IA64] make use of rcu_lock_target_domain_by_id()
X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~14100
X-Git-Url: https://dgit.raspbian.org/%22http:/www.example.com/cgi/%22https://%22%22/%22http:/www.example.com/cgi/%22https:/%22%22?a=commitdiff_plain;h=369b07383bc83169f222f916b079f15adad3a537;p=xen.git

[IA64] make use of rcu_lock_target_domain_by_id()

reduce code duplication by using rcu_lock_target_domain_by_id()

Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp>
---

diff --git a/xen/arch/ia64/vmx/vmx_hypercall.c b/xen/arch/ia64/vmx/vmx_hypercall.c
index d5ae9e731d..451074e450 100644
--- a/xen/arch/ia64/vmx/vmx_hypercall.c
+++ b/xen/arch/ia64/vmx/vmx_hypercall.c
@@ -50,16 +50,9 @@ static int hvmop_set_isa_irq_level(
     if ( op.isa_irq > 15 )
         return -EINVAL;
 
-    if ( op.domid == DOMID_SELF )
-        op.domid = current->domain->domain_id;
-
-    d = rcu_lock_domain_by_id(op.domid);
-    if ( d == NULL )
-        return -ESRCH;
-
-    rc = -EPERM;
-    if ( !IS_PRIV_FOR(current->domain, d) && d != current->domain )
-        goto out;
+    rc = rcu_lock_target_domain_by_id(op.domid, &d);
+    if ( rc != 0 )
+        return rc;
 
     rc = -EINVAL;
     if ( !is_hvm_domain(d) )
@@ -86,16 +79,9 @@ static int hvmop_set_pci_intx_level(
     if ( (op.domain > 0) || (op.bus > 0) || (op.device > 31) || (op.intx > 3) )
         return -EINVAL;
 
-    if ( op.domid == DOMID_SELF )
-        op.domid = current->domain->domain_id;
-
-    d = rcu_lock_domain_by_id(op.domid);
-    if ( d == NULL )
-        return -ESRCH;
-
-    rc = -EPERM;
-    if ( !IS_PRIV_FOR(current->domain, d) && d != current->domain )
-        goto out;
+    rc = rcu_lock_target_domain_by_id(op.domid, &d);
+    if ( rc != 0 )
+        return rc;
 
     rc = -EINVAL;
     if ( !is_hvm_domain(d) )
@@ -129,18 +115,9 @@ do_hvm_op(unsigned long op, XEN_GUEST_HANDLE(void) arg)
         if (a.index >= HVM_NR_PARAMS)
             return -EINVAL;
 
-        if (a.domid == DOMID_SELF) {
-            d = rcu_lock_current_domain();
-        }
-        else {
-            d = rcu_lock_domain_by_id(a.domid);
-            if (d == NULL)
-                return -ESRCH;
-            if (!IS_PRIV_FOR(current->domain, d)) {
-                rcu_unlock_domain(d);
-                return -EPERM;
-            }
-        }
+        rc = rcu_lock_target_domain_by_id(a.domid, &d);
+        if (rc != 0)
+            return rc;
 
         if (op == HVMOP_set_param) {
             struct vmx_ioreq_page *iorp;
@@ -213,20 +190,9 @@ do_hvm_op(unsigned long op, XEN_GUEST_HANDLE(void) arg)
         if ( copy_from_guest(&a, arg, 1) )
             return -EFAULT;
 
-        if ( a.domid == DOMID_SELF )
-        {
-            d = rcu_lock_current_domain();
-        }
-        else
-        {
-            if ( (d = rcu_lock_domain_by_id(a.domid)) == NULL )
-                return -ESRCH;
-            if ( !IS_PRIV_FOR(current->domain, d) )
-            {
-                rc = -EPERM;
-                goto param_fail3;
-            }
-        }
+        rc = rcu_lock_target_domain_by_id(a.domid, &d);
+        if ( rc != 0 )
+            break;
 
         rc = -EINVAL;
         if ( !is_hvm_domain(d) )
diff --git a/xen/arch/ia64/xen/mm.c b/xen/arch/ia64/xen/mm.c
index bc8c02db83..1a55f8da34 100644
--- a/xen/arch/ia64/xen/mm.c
+++ b/xen/arch/ia64/xen/mm.c
@@ -2815,6 +2815,7 @@ long
 arch_memory_op(int op, XEN_GUEST_HANDLE(void) arg)
 {
     struct page_info *page = NULL;
+    long rc;
 
     switch (op) {
     case XENMEM_add_to_physmap:
@@ -2826,16 +2827,9 @@ arch_memory_op(int op, XEN_GUEST_HANDLE(void) arg)
         if (copy_from_guest(&xatp, arg, 1))
             return -EFAULT;
 
-        if (xatp.domid == DOMID_SELF)
-            d = rcu_lock_current_domain();
-        else {
-            if ((d = rcu_lock_domain_by_id(xatp.domid)) == NULL)
-                return -ESRCH;
-            if (!IS_PRIV_FOR(current->domain,d)) {
-                rcu_lock_domain(d);
-                return -EPERM;
-            }
-        }
+        rc = rcu_lock_target_domain_by_id(xatp.domid, &d);
+        if (rc)
+            return rc;
 
         /* This hypercall is used for VT-i domain only */
         if (!is_hvm_domain(d)) {
@@ -2926,20 +2920,9 @@ arch_memory_op(int op, XEN_GUEST_HANDLE(void) arg)
         if ( copy_from_guest(&xrfp, arg, 1) )
             return -EFAULT;
 
-        if ( xrfp.domid == DOMID_SELF )
-        {
-            d = rcu_lock_current_domain();
-        }
-        else
-        {
-            if ( (d = rcu_lock_domain_by_id(xrfp.domid)) == NULL )
-                return -ESRCH;
-            if ( !IS_PRIV_FOR(current->domain, d) )
-            {
-                rcu_unlock_domain(d);
-                return -EPERM;
-            }
-        }
+        rc = rcu_lock_target_domain_by_id(xrfp.domid, &d);
+        if ( rc != 0 )
+            return rc;
 
         domain_lock(d);